Skip to content

Updated Error message for CALL with RETURN#887

Closed
aru-d-at wants to merge 1 commit into
apache:masterfrom
aru-d-at:master
Closed

Updated Error message for CALL with RETURN#887
aru-d-at wants to merge 1 commit into
apache:masterfrom
aru-d-at:master

Conversation

@aru-d-at

@aru-d-at aru-d-at commented May 2, 2023

Copy link
Copy Markdown
Contributor
  • ADDED Syntax Error on CALL func() RETURN rte

  • Adjusted CALL Regression Test expected ERROR message

The error message reflects missing rte declaration when running solo CALL function

Fixes issue #871

Checks:

  • Regression tests:
    Installcheck

@jrgemignani

jrgemignani commented May 2, 2023

Copy link
Copy Markdown
Contributor

@aru-d-at Please see my comment in the issue thread. This PR needs to address the root cause of the crash and produce an acceptable message or resolve the problem; either a syntax error or something about invalid usage. But ERROR: could not find rte is not acceptable as it is a problem in and of itself.

@aru-d-at aru-d-at closed this May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants